Skip to content

Remove chatty logs from default filter#21548

Merged
alice-i-cecile merged 5 commits intobevyengine:mainfrom
janhohenheim:better-log
Oct 14, 2025
Merged

Remove chatty logs from default filter#21548
alice-i-cecile merged 5 commits intobevyengine:mainfrom
janhohenheim:better-log

Conversation

@janhohenheim
Copy link
Member

@janhohenheim janhohenheim commented Oct 14, 2025 •
edited
Loading

Objective

Solution

  • Improve the default filter

Testing

  • Tested in a standalone project (Foxtrot)

@janhohenheim janhohenheim added D-Trivial Nice and easy! A great choice to get started with Bevy C-Usability A targeted quality-of-life change that makes Bevy easier to use A-Diagnostics Logging, crash handling, error reporting and performance analysis S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 14, 2025
@janhohenheim janhohenheim added this to the 0.18 milestone Oct 14, 2025
@alice-i-cecile
Copy link
Member

alice-i-cecile commented Oct 14, 2025 •
edited
Loading

This was previously raised upstream in pdeljanov/Symphonia#142, and closed as won't-fix. The latest Symphonia version improves things, but is still very chatty:

image

@janhohenheim
Copy link
Member Author

janhohenheim commented Oct 14, 2025 •
edited
Loading

NB: the above screenshot is truncated. The logs go on 🫠
image

dsgallups
dsgallups approved these changes Oct 14, 2025
View reviewed changes
Copy link
Contributor

@dsgallups dsgallups left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems desirable
ᗧ•••ᗣ•ᗣ••

@janhohenheim janhohenheim added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 14, 2025
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Oct 14, 2025
Merged via the queue into bevyengine:main with commit f7a6686 Oct 14, 2025
38 checks passed
mate-h pushed a commit to mate-h/bevy that referenced this pull request Oct 22, 2025
@janhohenheim @mate-h
Remove chatty logs from default filter (bevyengine#21548)
a0d3154 
# Objective

- Fixes bevyengine#16277
- The maintainers have already removed some annoying logs like the probe
one, but the format crates are still incredibly chatty
- Fixes bevyengine#14904
  - That's a completely benign warning that is annoying for Linux users

## Solution

- Improve the default filter

## Testing

- Tested in a standalone project (Foxtrot)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alice-i-cecile alice-i-cecile alice-i-cecile approved these changes

+1 more reviewer

@dsgallups dsgallups dsgallups approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

A-Diagnostics Logging, crash handling, error reporting and performance analysis C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Milestone

0.18

Development

Successfully merging this pull request may close these issues.

Remove excessive logging when using symphonia features like AudioBundle calloop warning

3 participants

@janhohenheim @alice-i-cecile @dsgallups